@rahul-tuli We can talk more about this somewhere else. I'd rather keep the/a dense compressor and get rid of the nullable property of compressors. This would make a lot of code much cleaner.

if compressor is not None:
  compressor.compress(model)

if compressor is not None:
  compressor.decompress(model)

is a lot more annoying to check every time you want to use a compressor programmatically, rather than a simply narrow type

compressor.compress(model)
compressor.decompress(model)

@rahul-tuli We can talk more about this somewhere else. I'd rather keep the/a dense compressor and get rid of the nullable property of compressors. This would make a lot of code much cleaner.

if compressor is not None:

  compressor.compress(model)



if compressor is not None:

  compressor.decompress(model)

is a lot more annoying to check every time you want to use a compressor programmatically, rather than a simply narrow type

compressor.compress(model)

compressor.decompress(model)

Sounds good @kylesayrs, could you create a ticket/issue so we remember to revisit this?

Approving the PR!